feat: unify annotation system to server-backed review API - #16
Merged
Conversation
…y#15) Replace the localStorage-based annotation system with the existing server-backed review API so comments persist in status.yaml and are visible to all users across both chapter pages and the dependency graph. - Rewrite annotations.js (~580→200 lines): fetch from GET /api/status, post via POST /api/review, scan _thmwrapper elements for indicators - Simplify annotations.css (~345→200 lines): remove edit/resolve/toolbar styles, add topic selector and feedback message styles - Fix "Complete (definition)" duplication bug in dep_graph: use .completion-label class selector instead of unreliable inline style attribute matching - Add Docker deployment: Dockerfile (multi-stage build), docker-compose, .dockerignore - Add project homepage (homepage/index.html) Closes surenny#15 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
status.yaml，多人可见，章节页与依赖图共享同一数据源GET /api/status获取评论、POST /api/review提交评论，扫描_thmwrapper元素显示指示器updateButtons()改用.completion-label类选择器替代不可靠的内联 style 属性匹配，解决每次 Send 后 "Complete (definition)" 重复叠加的问题homepage/index.htmlTest plan
docker compose up -d --build构建并启动容器sec-prereq.html），确认定理旁有 ✍ 指示器🤖 Generated with Claude Code